Skip to content

Conversation

mariadb-stefangenov
Copy link
Collaborator

No description provided.

For the plugins specifically, Have not listed the old `.2` version
as that does not include the full suite of plugins (missing hashicorp)
Copy link
Member

@mariadb-mmontes mariadb-mmontes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking good! Left some comments.

This commit is a work in progress, just pushing some changes while @ KCD
… docs to add ENV var option

There is a log message that says we can use `VAULT_TOKEN` if `hashicorp-key-management-token` is not provided, but wasn't part of the
documentation.

The example for the operator uses that approach. Updated to add policy and added examples how things will look like.
Copy link
Member

@mariadb-mmontes mariadb-mmontes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Left some improvements.

I can confirm that works as intended.

@mariadb-mmontes mariadb-mmontes self-requested a review September 19, 2025 14:36
@mariadb-mmontes mariadb-mmontes marked this pull request as ready for review September 19, 2025 14:39
@mariadb-mmontes mariadb-mmontes changed the title (WIP/DRAFT) Docs/hashicorp key management.md Docs/hashicorp key management.md Sep 19, 2025
@mariadb-stefangenov mariadb-stefangenov merged commit fc84415 into main Sep 19, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants